Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

fix: redirect debug output to stderr in talos-embed-config hook#48

Merged
jmgilman merged 1 commit into
masterfrom
fix/talos-hook-stdout
Dec 29, 2025
Merged

fix: redirect debug output to stderr in talos-embed-config hook#48
jmgilman merged 1 commit into
masterfrom
fix/talos-hook-stdout

Conversation

@jmgilman
Copy link
Copy Markdown
Collaborator

Summary

  • Redirects debug messages to stderr in generate_config function
  • Only the config file path is output to stdout for capture

Context

The generate_config function was printing messages like "Generating machine configuration..." to stdout, which got captured into the config_file variable along with talhelper output. This caused the subsequent cp command to fail.

Error:

cp: cannot stat 'Generating machine configuration...'$'\n''generated config for cp-1...'

Test plan

  • Merge and manually trigger workflow

🤖 Generated with Claude Code

The generate_config function was printing messages to stdout, which
got captured along with the config file path. This caused the cp
command to fail with the debug output as the source file path.

Now only the config file path is output to stdout; all other messages
go to stderr for display.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@jmgilman jmgilman merged commit 3d79ed2 into master Dec 29, 2025
1 check passed
@jmgilman jmgilman deleted the fix/talos-hook-stdout branch December 29, 2025 05:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant